Skip to content

Add dronpablo.xplanecom X-Plane COM Radio plugin - #30

Closed
pablovuela wants to merge 3 commits into
OpenActionAPI:mainfrom
pablovuela:main
Closed

pablovuela wants to merge 3 commits into
OpenActionAPI:mainfrom
pablovuela:main

Conversation

@pablovuela

Copy link
Copy Markdown

No description provided.

@pablovuela

Copy link
Copy Markdown
Author

Hello, this plugin integrates with Xplane flight simulator providing interface for radio communications, let me know if the Pull Request is valid or any changes must be done, thank you.

@nekename

Copy link
Copy Markdown
Member

I just commented on #28 that I accept LLM-assisted plugin contributions but not from people with no idea what they're doing. Your repo has no plugin release artifact and is full of __pycache__ / .pyc files...

@pablovuela

Copy link
Copy Markdown
Author

haha, you are right I have no much idea what I'm doing, but comme on dont be mean we all here are giving our time for free, the plugin works perfectly fine, I will just fix that and ask you again, be patient with the noobs.

@nekename

nekename commented Jun 25, 2026

Copy link
Copy Markdown
Member

I'd be super happy to help anyone with no programming experience develop a plugin. Accepting it into a curated catalogue, though, requires it to meet certain quality standards, because these plugins are consumed by thousands of users.

@pablovuela

Copy link
Copy Markdown
Author

Hello there, whenever you can please check if I fixed everything or I'm still missing something, thank you!

@nekename

Copy link
Copy Markdown
Member

Why did you add a download_url, locking it to a specific release as well?

@pablovuela

pablovuela commented Jun 29, 2026

Copy link
Copy Markdown
Author

You are right there is no need for that

@pablovuela

Copy link
Copy Markdown
Author

Hello, is there anything else I should fix before merging?

@nekename

nekename commented Jul 2, 2026

Copy link
Copy Markdown
Member

I haven't had time to check this or the other open submissions yet, sorry. I'm quite busy and in my limited free time, working on some new projects.

@nekename

Copy link
Copy Markdown
Member

OK, I've had time to look at this now, and unfortunately I won't be able to accept this. You haven't used a valid reverse-DNS ID for your plugin, and regardless, this plugin runs with /usr/bin/python3 instead of a bundled-Python solution making it incompatible with Windows at the very least, if not some Linux distros as well.

@nekename nekename closed this Sep 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants